-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move docker image to break circular dependency #3002
Conversation
Just getting the PR started to see what happens with the build. |
I've moved the images to https://github.com/vvcephei/ksql-images/tree/5.1.x , but there are a couple of problems... The big blocker is that I can't create a PR to create a new branch. I need a committer to create the 5.1.x branch on https://github.com/confluentinc/ksql-images @rodesai , can you help out? |
Also, I'm not sure why the build failed. It failed on Twistlock being unable to find Ideas? |
@vvcephei you are already an admin for ksql-images.
The twistlock is something the @confluentinc/tools team merged a few days ago. Can someone from there comment on this? |
Did you check if the image was still there? There's always a chance something caused it to be cleaned up during the build. Didn't check thoroughly, but pull failed for me. Also, /cc @xiangxin72 who might have additional insight. |
@vvcephei Can you provide me the logs that twistlock failed? It may caused by either that the image is cleaned or wrong tags. |
@xli1996 , thanks for taking a look. Here's the logs: https://jenkins.confluent.io/job/confluentinc-pr/job/ksql/job/PR-3002/3/consoleText Fair warning, I am not familiar with this build at all, so we may need to do some sleuthing to figure how how it's supposed to go. |
I don't see major difference between #2059 and this one, and the master branch works fine. The logs claims that the image is built successfully, maybe I will try disable twistlock temporary with |
The twistlock issue seems to be flaky and only happen in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merging to fix the build. |
Hey, thanks, all. Sorry, I haven't checked in since your comments. I'll follow up and make sure that the image build gets added to the ksql-images repo. |
The docker image build was moved out of this repo in the 5.2.x branch to break a circular dependency, but the circular dependency was already present in the 5.1.x branch. This PR corrects the issue.